Skip to content

Fix FomanticUI#2854

Open
samialfattani wants to merge 48 commits into
pallets-eco:masterfrom
samialfattani:fix/fomantic
Open

Fix FomanticUI#2854
samialfattani wants to merge 48 commits into
pallets-eco:masterfrom
samialfattani:fix/fomantic

Conversation

@samialfattani
Copy link
Copy Markdown
Contributor

@samialfattani samialfattani commented Apr 1, 2026

This PR enhances Fomantic UI #2643 to be compatible with latest changes.

List of enhancements

Issues still need to be fixed

princerb and others added 30 commits July 25, 2025 18:01
@samialfattani samialfattani marked this pull request as ready for review April 1, 2026 20:23
@ElLorans
Copy link
Copy Markdown
Contributor

ElLorans commented Apr 7, 2026

Thanks!

  1. Why do we need docker to test this?
  2. Can we remove the fix the CSRF token on delete files and directories #2829 since it's a different topic?

@samialfattani
Copy link
Copy Markdown
Contributor Author

  • Why do we need docker to test this?

i reviewed the source code but i couldn't find any docker refernce in the change. plus, i can run the examples/fomanticui normally like all other examples without need to docker.

Can we remove the #2829 since it's a different topic?

it was a bug in bootstrap4, i think if we are adding new UI we should avoid old mistakes. anyway the CSRF part within HTML pages and we still did not touch any *.py files.

@ElLorans ElLorans added the theme label May 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants